Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readme updates #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tpietruszka
Copy link

Got here through the Mogrifier paper. Great read, and the 'case for small scale' is really refreshing. I am surprised to see no activity here.

Updated a few broken links in READMEs

Also, installing tensorflow-probability-gpu=1.15 did not work (package not found), but I have found some claims that it is now sufficient to install tensorflow-probability. Training seems to work (at least for a few "turns"), without the -gpu.

@melisgl
Copy link
Collaborator

melisgl commented Mar 2, 2020

Thank you. I'll merge this as soon as I figure out how to include incorporate the changes correctly into the internal codebase from which this repo is exported.

@jkkummerfeld
Copy link

Not sure what I did differently, but I found the code didn't work with the latest version of tensorflow-probability. Instead I solved this issue with he following:

    pip install --upgrade --user "tensorflow<2" "tensorflow_probability<0.9"
    pip install dm-sonnet==1.36

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants